Skip to content

refactor plugin to use OpenGL fbo rendering #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

refactor plugin to use OpenGL fbo rendering #17

wants to merge 21 commits into from

Conversation

mbaetgen-wup
Copy link

@mbaetgen-wup mbaetgen-wup commented Oct 6, 2025 •
edited
Loading

Rendering improvements

  • use OpenGL render-to-texture with Frame Buffer Objects (FBO)
  • send GL textures downstream - video/x-raw(memory:GLMemory)
  • use buffer pool for managing GL textures
  • optimized GL thread usage
  • real-time rendering optimizations and real-time QoS
  • offload rendering tasks from chain function to render buffer (sync and async rendering)

General fixes and improvements

  • improved GST event handling (segments, QoS)
  • fix some locking issues and race conditions
  • fix sync issues with pipeline clock
  • auto-detect offline or real-time rendering
  • re-usable projectM/gst integration functions
  • naming cleanup
  • more docs

Requires projectM 4.2 (not yet released, current master branch version)

Test status
Testing is on-going, all known bugs are fixed.

Linux: Real-time rendering stable (over 19h) / Offline rendering stable
Windows: ?

@hack-s hack-s mentioned this pull request Oct 8, 2025
Open
rootnotez
rootnotez reviewed Oct 9, 2025
View reviewed changes
Copy link

@rootnotez rootnotez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the /examples directory need to be created? It is confusing with the /test directory, and makes it confusing where the audio/video samples are kept.

@mbaetgen-wup
Copy link
Author

that's a good point. The example itself may also not be that useful in it's current state. I will remove it from this PR and add it again in a separate PR later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@rootnotez rootnotez rootnotez left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mbaetgen-wup @rootnotez